Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

프로필 바텀 시트 UI 구현 #744

Merged
merged 10 commits into from
Oct 24, 2024
Merged

프로필 바텀 시트 UI 구현 #744

merged 10 commits into from
Oct 24, 2024

Conversation

cys4585
Copy link
Contributor

@cys4585 cys4585 commented Oct 23, 2024

PR의 목적이 무엇인가요?

다락방 멤버들의 프로필을 조회할 수 있는 바텀 시트 UI를 구현했어요.

이슈 ID는 무엇인가요?

설명

image

  1. 바텀 시트 컴포넌트를 구현했습니다.
  2. 바텀 시트를 관리하는 커스텀 훅을 구현했습니다.
  3. 모임 디테일, 룰렛 디테일, 채팅 룸, 채팅 사이드 바, 다락방 멤버 목록에서 멤버 카드 UI를 클릭하면 프로필 바텀 시트를 띄웁니다.
  4. 사이즈: small, medium, large, full 대응 가능하도록 구현했지만, 기본적으로 컨텐츠 양에 따라 사이즈가 확장됩니다.
  5. 드래그 처리 및 애니메이션을 적용했습니다.
  6. 참여자, 다락방 멤버 등의 API 스펙에 darakbangMemberId가 추가됨에 따라 type 정의를 수정했고, 해당 타입을 쓰는 사용처도 수정했습니다. 문제가 없는 걸 확인했지만, 크로스 체크 해주면 감사하겠습니다.
  7. API 호출 시점이나 안정성, 렌더링 낭비 측면에 대해 큰 고민을 하지는 못했어요.
  8. 프로필 이미지는 정확하게 확인을 하지 못했습니다. 우선 다른 컴포넌트를 보고 넣어놨는데, 배포하면서 같이 확인해보면 좋겠습니다.

질문 혹은 공유 사항 (Optional)

  • 데모 마감으로 인해 급하게 개발하느라 css는 인라인으로 일단 때려 넣었습니다. 추후 수정하도록 하겠습니다....
  • 바텀시트를 드래그로 닫을 수 있도록 하는 것도 구현하지 못했습니다. 추후 반영하겠습니다.

@cys4585 cys4585 added FE 프론트엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현) labels Oct 23, 2024
@cys4585 cys4585 self-assigned this Oct 23, 2024
@cys4585 cys4585 requested a review from jaeml06 October 23, 2024 13:41
@jaeml06 jaeml06 merged commit 17eab02 into develop-frontend Oct 24, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants